Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create tests for APG design pattern example: Tri-State Checkbox #330

Merged
merged 27 commits into from
Mar 25, 2021

Conversation

jscholes
Copy link
Contributor

@jscholes jscholes commented Nov 10, 2020

Preview Tests

Resolves #322.

jscholes and others added 13 commits November 10, 2020 14:43
The following CSS and JS references have been removed:

* `../../css/core.css`
* `../../js/examples.js`
* `../../js/highlight.pack.js`
* `../../js/app.js`

The following references have been modified to match the directory structure of code once added to the aria-at repo:

* `../css/checkbox.css` -> `css/checkbox.css`

The reference to https://www.w3.org/StyleSheets/TR/2016/base.css has been maintained.
… setup scripts

The links are implemented with an `href` attribute which references their own ID.  This pattern creates links which are valid (according to the W3C's validator) but don't move the keyboard focus if accidentally activated by a tester.
@jscholes jscholes marked this pull request as ready for review November 12, 2020 20:08
@jscholes jscholes requested a review from IsaDC November 12, 2020 20:08
@jscholes
Copy link
Contributor Author

Review notes:

  1. There are no buttons to open the example on the review page, as these weren't included on the page template at the time this PR was filed. Buttons to open test pages were added in commit ec299fc.
  2. The example in the reference directory isn't timestamped, again because the script and pattern for this were added later than this PR (in 4d77ba2).

Both of the above can easily be fixed by merging master into this tests branch, adding the timestamped directory and then allowing the test pages to be regenerated. But there are currently merge conflicts in files that haven't been manually touched during creation of this PR, such as the unrelated menubar editor tests.

@mcking65 mcking65 merged commit e7212c4 into master Mar 25, 2021
@mcking65 mcking65 deleted the tests/checkbox-tri-state branch March 25, 2021 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create tests for APG design pattern example: Tri-State Checkbox
3 participants